Skip to content

feat(agent): add jwtsource package for reading projected SA tokens - #818

Open
roeezis wants to merge 1 commit into
jetstack:masterfrom
roeezis:split/01-jwtsource
Open

feat(agent): add jwtsource package for reading projected SA tokens#818
roeezis wants to merge 1 commit into
jetstack:masterfrom
roeezis:split/01-jwtsource

Conversation

@roeezis

@roeezis roeezis commented Aug 23, 2026

Copy link
Copy Markdown

Summary

Part 1 of the SMS/Conjur JWT authentication series (split out of #817 for reviewability).

Introduces a small, isolated interface for reading a JWT from a file path — the first piece of the upcoming Conjur JWT authentication path. Nothing else depends on it in this PR; the Conjur client that uses it lands later in the stack.

Test plan

  • go build ./...
  • go test ./internal/cyberark/jwtsource/...

Comment thread internal/cyberark/jwtsource/jwtsource_test.go Outdated
Introduces a small, isolated interface for reading a JWT from a file
path — the first piece of the upcoming Conjur JWT authentication path,
split out on its own since nothing else in this PR depends on it yet.
@roeezis
roeezis force-pushed the split/01-jwtsource branch from f441e36 to ffa53d1 Compare August 24, 2026 11:45
@roeezis

roeezis commented Aug 24, 2026

Copy link
Copy Markdown
Author

Fixed — switched both tests to t.Context().

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants